Skip to content
This repository was archived by the owner on May 29, 2026. It is now read-only.

ci: install OpenRegister development (runtime-schema-API + lifecycle) — closes #29#30

Merged
rubenvdlinde merged 1 commit into
developmentfrom
fix/ci-pin-openregister-ref
May 12, 2026
Merged

ci: install OpenRegister development (runtime-schema-API + lifecycle) — closes #29#30
rubenvdlinde merged 1 commit into
developmentfrom
fix/ci-pin-openregister-ref

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

Closes #29. OpenBuilt's CI installs OpenRegister as an additional-apps dep for the PHPUnit-on-NC-server + Newman jobs; it was pinned to OR main, which predates both the runtime-schema-API (auto-create a Register on an application-type config import — OR PR #1464) and the lifecycle engine (ObjectTransitionedEvent). Without the former, InitializeSettings can't create the openbuilt register on app:enable, so SeedHelloWorld/SeedApplicationTemplates fail and every Newman assertion fails; without the latter, the Newman publish step's transition call has no endpoint. Points it at OR development (which now has both — and the Cannot redeclare Organisation::$mail regression that was blocking OR development's app-enable was fixed in OpenRegister PR #1499).

OpenBuilt's CI installs OpenRegister as an `additional-apps` dependency
for the PHPUnit-on-NC-server + Newman jobs. It was pinned to OR `main`,
which predates both the runtime-schema-API (auto-create a Register on an
application-type configuration import — OR PR #1464) and the lifecycle
engine (ObjectTransitionedEvent). Without the former, `InitializeSettings`
can't create the `openbuilt` register on `app:enable`, so `SeedHelloWorld`
/ `SeedApplicationTemplates` fail and every Newman assertion fails; without
the latter, the Newman publish step's transition call has no endpoint.

Point it at OR `development`, which now has both (and the
`Cannot redeclare Organisation::$mail` regression that was blocking its
app-enable was fixed in OR #1499). Closes openbuilt#29.
@github-actions

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/openbuilt @ f33a15c

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
composer ✅ 100/100
npm ✅ 432/432
PHPUnit
Newman
Playwright ⏭️

Coverage: 0% (0/19 statements)


Quality workflow — 2026-05-12 12:27 UTC

Download the full PDF report from the workflow artifacts.

@rubenvdlinde rubenvdlinde merged commit 88b0f5c into development May 12, 2026
24 of 25 checks passed
@rubenvdlinde rubenvdlinde deleted the fix/ci-pin-openregister-ref branch May 12, 2026 12:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant